Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: Update cors.md, Explanation of how routing HTTP method configuration affects browser behavior when the CORS plugin is activated #11602

Open
wants to merge 1 commit into
base: release/3.10
Choose a base branch
from

Conversation

cylinder-lee-cn
Copy link

@cylinder-lee-cn cylinder-lee-cn commented Sep 25, 2024

对路由中http方法的配置导致cros插件生效后浏览器无法按照预期工作的说明

Description

对路由中http方法的配置导致cros插件生效后浏览器无法按照预期工作的说明

Checklist

  • I have explained the need for this PR and the problem it solves
  • I have explained the changes or the new features added to this PR
  • I have added tests corresponding to this change
  • I have updated the documentation to reflect this change
  • I have verified that this change is backward compatible (If not, please discuss on the APISIX mailing list first)

对路由中http方法的配置导致cros插件生效后浏览器无法按照预期工作的说明
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. doc Documentation things labels Sep 25, 2024
@cylinder-lee-cn
Copy link
Author

update commit

@cylinder-lee-cn cylinder-lee-cn changed the title Update cors.md, Explanation of how routing HTTP method configuration affects browser behavior when the CORS plugin is activated docs: Update cors.md, Explanation of how routing HTTP method configuration affects browser behavior when the CORS plugin is activated Sep 25, 2024
@cylinder-lee-cn
Copy link
Author

update title

@kayx23
Copy link
Member

kayx23 commented Oct 28, 2024

Hi, which browser did you use that didn't enable OPTIONS by default?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Documentation things size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants